Skip to content

Add option to compress out non-dEdx info in TrackQA table #15045

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
ddobrigk wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Add option to compress out non-dEdx info in TrackQA table #15045

ddobrigk wants to merge 5 commits into from
+19 −16

Conversation

@ddobrigk
Copy link
Contributor

@ddobrigk ddobrigk commented Feb 10, 2026

This PR, coupled with #15010, allows for the possibility of keeping Qmax and Qtot (plus the normalization quantity tpcdEdxNorm) for all global tracks while compressing out the remainder 60-70% of the data volume. This effectively decouples the development of dEdx corrections and the development of ITSTPC tracking studies that have no reason for actually being coupled in terms of resource usage.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 10, 2026

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@ddobrigk ddobrigk marked this pull request as ready for review February 10, 2026 11:48
@ddobrigk ddobrigk requested a review from a team as a code owner February 10, 2026 11:48
@alibuild alibuild mentioned this pull request Feb 10, 2026
Merged
@ddobrigk
Merge pull request #40 from alibuild/alibot-cleanup-15045
b291347 
Please consider the following formatting changes to AliceO2Group#15045
@ddobrigk
Copy link
Contributor Author

ddobrigk commented Feb 10, 2026

@shahor02 @f3sch note that I made one more change to this PR in which, in case the "keep globals" is enabled for TrackQA saving, also TPC-only tracks that are used by the svertexer (photons) have their TrackQA entries kept. This will allow us to use any improved TPC procedure also for photon analyses. The impact in size is at the per-mille level, since actual TPC-only tracks used by the svertexer are only a few. I've also tested this PR and everything seems to work fine. Please take a look and jump up in case you spot something unexpected or otherwise suspicious. Thanks a lot!

shahor02
shahor02 approved these changes Feb 10, 2026
View reviewed changes
Copy link
Collaborator

@shahor02 shahor02 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shahor02 shahor02 shahor02 approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ddobrigk @shahor02 @alibuild